Skip to content

Conversation

@marcleblanc2
Copy link
Contributor

Checklist

Test plan

Ran helm template, helm lint, and helm unittest, all passed

@marcleblanc2 marcleblanc2 requested a review from a team June 26, 2025 22:09
@marcleblanc2 marcleblanc2 enabled auto-merge (squash) June 26, 2025 22:09
@marcleblanc2 marcleblanc2 disabled auto-merge June 26, 2025 22:14
@marcleblanc2 marcleblanc2 merged commit 2e145ab into main Jun 26, 2025
7 checks passed
@marcleblanc2 marcleblanc2 deleted the marc/config-to-remove-role-binding branch June 26, 2025 22:19
marcleblanc2 added a commit that referenced this pull request Jul 3, 2025
… to be disabled (#718)

### Checklist

- [x] Follow the [manual testing
process](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/TEST.md)
- [ ] Update
[changelog](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/charts/sourcegraph/CHANGELOG.md)
- [ ] Update [Kubernetes update
doc](https://docs.sourcegraph.com/admin/updates/kubernetes)

### Test plan
Ran `helm template`, `helm lint`, and `helm unittest`, all passed

 <br> Backport 2e145ab from #704

Co-authored-by: Marc <[email protected]>
enriquegh pushed a commit that referenced this pull request Jul 10, 2025
…704)

### Checklist

- [x] Follow the [manual testing
process](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/TEST.md)
- [ ] Update
[changelog](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/charts/sourcegraph/CHANGELOG.md)
- [ ] Update [Kubernetes update
doc](https://docs.sourcegraph.com/admin/updates/kubernetes)

### Test plan
Ran `helm template`, `helm lint`, and `helm unittest`, all passed
<!--
As part of SOC2/GN-104 and SOC2/GN-105 requirements, all pull requests
are REQUIRED to
provide a "test plan". A test plan is a loose explanation of what you
have done or
implemented to test this, as outlined in our Testing principles and
guidelines:

https://docs.sourcegraph.com/dev/background-information/testing_principles
  Write your test plan here after the "Test plan" header.
-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants